GtkEntry: Remove recompute idle

Just do it directly

GtkEntry: Remove unused fields

Adwaita: some OSD rationalization

taking care of https://bugzilla.gnome.org/show_bug.cgi?id=732528 in
the process, any toolbar.osd with a positional style class attached
gets squared corners now.

Adwaita: scale with marks styling

I'm really sorry for the unlucky ones who would have to read that
code...

Adwaita: added back scale marks style

tests: Rework some foreigndrawing code

Instead of specifying the full path everytime, just specify the parent
style context and then use its path.

Update gtk_drag_highlight() docs

What they said was no longer true.

Simplify foreign drawing example a bit

Adwaita: scale fill style

GtkAppChooserWidget: wrap the label, use a standard icon

Let the label wrap if needed, that's required in some locales, use an
icon which is available in the theme

https://bugzilla.gnome.org/show_bug.cgi?id=758908

Adwaita: rename scale sass templates to be more specific

Adwaita: added back style for scales on selected list rows

Adwaita: first stab at GtkScale and GtkProgressBar rewrite

...scale has marks stuff missing, coming soon.

window: Deprecate gtk_window_resize_to_geometry()

And make it not do anything anymore.

Fixes erratic resizes of gnome-terminal.

https://bugzilla.gnome.org/show_bug.cgi?id=757282

gtkwindow: Document further resize with csd

Applying the client-side decorations in the configure routine greatly
increases the chances of having the right size for the GtkHEaderBar and
border shadows.

Yet, it may be possible that these sizes change at a later point in
time, if for example the GtkHeaderBar grows in height while adding new
controls.

Mention this possible pitfall in the documentation for
gtk_window_resize().

https://bugzilla.gnome.org/show_bug.cgi?id=756618

entry: Use regular drag highlight

The entry code passes GTK_DEST_DEFAULT_HIGHLIGHT when setting
up the drop target, but that is ineffective because of the
custom drag_motion implementation. Instead, call
gtk_drag_[un]hightlight ourselves.

Use :dnd pseudoclass for drag highlighting

Instead of a ::draw handler with a hardcoded outline,
use CSS for drawing the highlight.

css: Add a :dnd pseudoclass

This will be used for drag highlighting.

Add a selected text example

Make foreign drawing example work

Set up multiple style contexts to get inheritance right.

Add another hand-drawn scrollbar

This one is supposed to show up in active state, but doesn't.

Add an example of foreign drawing

This uses the gtk_render apis with a style context constructed
from a style path.

css: Introduct -gtk-icon-palette

This borrows heavily from the CSS4 fonts draft's font-palette, currently
found at https://drafts.csswg.org/css-fonts-4/#font-palette-control

The palette is mainly meant to trigger invalidations when colors used for
symbolic icons change, to potentially allow extending supported colors
in symbolic icons and to recolor all colors of a symbolic icon, not just
the main one.

The syntax for the property goes like this:
Name:        -gtk-icon-palette
Value:       default | name <color> [ , name <color> ]*
Initial:     default
Applies to:  all elements with icons
Inherited:   yes
Animatable:  yes, each color animated separately

The property defines a list of named colors to be used when looking up
icons. If a name is not defined, the value of the current "color"
property is used. Which names are relevant depends on the icons in use.
Currently symbolic icons make use of the names "success", "warning" and
"error".

"default" is the current behavior of the GTK when coloring symbolic
icons and is equal to the string
  success @success_color, warning @warning_color, error @error_color

Animation is crudely implemented by animating colors that are in both
palettes that are animated and otherwise keeping the color from the
palette that defined it. Note that this can cause a sharp cut at the
beginning or end of the animation when the color goes away and will
therefore be replaced with the color property.

You can see an example of animations at
http://gfycat.com/CautiousPeacefulIaerismetalmark

Updated Spanish translation

GtkRange: Return proper constants in signal handlers

GtkRange: Remove recalc_marks field

And instead recalculate the marks on demand, i.e. whenever we were
previously setting recalc_marks to TRUE.

GtkRange: Use G_MININT as default mouse position, not -1

Otherwise we're getting MOUSE_WIDGET as mouse position for unmapped
GtkRanges.

GtkScale: Add missing nullable annotations

Adwaita: button.color fix

border-radius: 0 for colorswatch overlay as well.

Adwaita: vertically linked insensitive entries

Brighter border in that case as well.

dnd: Add gtk_image_set_from_definition()

... and use it in the DND code, thereby getting rid of the icon helper.

dnd: No need to reset the icon

When we start a drag cancel animation, we can just keep the existing
window. The reset was only necessary to convert from cursor icon to
window and we removed the cursor handling.

image: Remove unused code

I wonder how long that stuff has existed and not been used...

image: Split out a function

imagedefinition: Assert when trying to free empty definition

Empty definitions are not allocated on a heap, so doing a g_free() on
it would crash anyway.

cssprovider: Pass NULL unconditionally

We know that parent is NULL at this point.

Also, fix indentation.

gtkwindow: apply CSD in configure size request

Just like we did for the default size, that reduces the chances of
having the headerbar missing or wrongly sized when computing the client
side decorations controls.

https://bugzilla.gnome.org/show_bug.cgi?id=756618

popover: Make position property not CONSTRUCT

There is no good reason for it, and it causes problems
with widget templates.

popover: Port to g_object_install_properties

And use the pspecs for notification.

dnd: Drop an unused argument

dnd: Some simplifications

We can now use a GtkImage instead of toplevel with
a custom draw callback and a specially prepared cairo
pattern.

Drop gdk_wayland_drag_context_get_dnd_window

This backend-specific and libgtk-only API is now unused,
so drop it.

dnd: Prevent black background in icons

DND: Drop support for using rgba cursors as icons

This adds a lot of complication to the code, and is not really
a big win, since still need to support icon windows anyway.

dnd: Fix issues with drag icons under Wayland

The Wayland dnd surface must remain in place until the drag
is over. Setting it directly as the hardcoded window of the
widget we construct carries the danger that it might get
destroyed prematurely, e.g. when the application calls
gtk_drag_set_icon_name more than once and we recreate the
widget.

Instead, create a dedicated toplevel, and reparent the widget
into it. To keep the code simple, we use the same approach
under X11 as well, and make it the responsibility of the
GDK dnd code to keep the window position updated. We already
pass the current pointer position to gdk_drag_motion, which
makes this very easy.

As a side-effect of these changes, it is now possible to use
non-toplevel widgets as drag icons.

https://bugzilla.gnome.org/show_bug.cgi?id=748763

Add gdk_drag_context_get_drag_window

This makes gdk_wayland_drag_context_get_dnd_window
backend-independent API and adds an implementation
for X11.

widget: Don't update pango context in state_flags_changed

It only needs to be updated in style_updated, and we do it there.

Chain up in state_flags_changed

When introducing handlers for state_flags_changed in the node
transitions, chaining up was forgotten.

css: Add ability to specify icontheme in CSS

-gtk-icontheme: "oxygen" works now.

This is a very crude implementation. It's meant for testing only.

window: Listen to icon theme changes on CSS

No need to connect a signal handler to the icon theme, we get notified
via CSS.

css: Query icon theme from style, not from settings

No need to look at the settings when the CSS has a property for the icon
theme.

spinner: Active spinners should be :checked, not :active

If that sounds confusing, it's because GTK and CSS can sometimes not
agree on naming.

:active for CSS means that a button is currently pressed on an element.
And that is clearly not the case for spinning spinners.

stylecontext: Don't emit state warning for transient nodes

It's fine to call set_state() on those.

https://bugzilla.gnome.org/show_bug.cgi?id=758930

css: Rename -gtk-image-effect to -gtk-icon-effect

This is a property for icons, so we should name it as such.

Stop using gtk_icon_size_lookup_for_settings()

Use the equivalent gtk_icon_size_lookup() instead.

iconhelper: Stop tracking state

The iconfactory code doesn't use it anymore, so we don't need to track
it either.

iconfactory: Make state depend on image-effect

This removes the dependency on state, which should be used for selection
CSS styles, not for actually applying them.

And image-effect does exactly what we want already, so we can start
using it.

iconhelper: Remove _gtk_icon_helper_ensure_pixbuf()

The code isn't used anymore. Everything uses
gtk_icon_helper_load_surface() now.

statusicon: Use gtk_icon_helper_load_surface()

This is in preparation for the next commit.

dnd: Add gtk_drag_set_icon_definition()

... and use it for entry icons.

As a side effect, icons dragged from entries will now resize to DND
size.

imagedefinition: Remove icon-size

The size of icons is a property that is relevant to who is rendering the
icon, not to the icon itself.

Example: Starting a DND operation from an entry icon should cause the
icon to resize (from the entr icon's size to the DND icon size).

dnd: Rewrite iconhelper code

It now uses GtkImageDefinition instead.

entry: Use gtk_icon_helper_load_surface()

This is the first step towards getting rid of pixbuf loading code in the
icon helper.

iconhelper: Add a scale argument to gtk_icon_helper_load_surface()

It will be used in the next patch.

iconhelper: Refactor ensure_surface()

Make gtk_icon_helper_ensure_surface() a private function that just
ensures the surface was loaded.

Add gtk_icon_helper_load_surface() that is called by the above function
and the dnd code to actually load the surface.

iconhelper: Store surface in ensure_surface()

Do not assign it in the functions that actually load the surface. Make
those just return the surface.

iconhelper: Don't store surface size

Instead, compute it on demand.

This avoids having to cache it in lots of places and simplifies code.

iconhelper: Move invalidation check

Just do the invalidation check once, there's no need to do it in every
branch of the switch.

Also remove useless checks: These functions will not be called if we
already have a rendered surface.

dnd: Use surface for dnd icon instead of pixbuf

iconhelper: Move invalidation check

Just do the invalidation check once, there's no need to do it in every
branch of the switch.

Also remove useless checks: These functions will not be called if we
already have a rendered surface.

iconhelper: Improve size determination for surfaces

Instead of just working for image surface, this should now work for all
bounded surfaces.

Test included.

iconhelper: Move size computation

We do a switch on the image type, so compute the size for a certain
image type right there.

image: queue_resize() after changing icon-size property

dragsource: Store an itemdefinition, not an iconhelper

The icon is never rendered, so there's no need for the extra iconhelper
code.

imagedefinition: Split out from icon helper

The image definition is supposed to hold the description about the image
to be displayed. The icon helper actually does caching and tracks
changes.

window: Remove suspicious branch

It seems this branch is not needed anymore. It was originally added in
1999 to support gtk_widget_realize(), but all those reasons seem
obsolete today.
Instead just call gtk_widget_realize().

If you end up at this commit when bisecting:
There is no bug that made me remove this code, it was purely meant to be
cleanup / dead code removal. I seem to have introduced a new bug or
bisecting wouldn't have let you here. So it seems we should just revert
this commit.

Revert "gdk_pixbuf_get_from_window: honor device scale"

This reverts commit 657a43e54e7721fced8ef0e6808271838f14697f.

The commit breaks the assumptions about the arguments in both
gdk_pixbuf_get_from_window() and gdk_pixbuf_get_from_surface().

https://bugzilla.gnome.org/show_bug.cgi?id=757147

wayland: Remove unused variable

Poor gcc was unhappy.

popover: Make it possible to constrain to toplevel

Under X11, popovers are always constrained to the toplevel
window. Under Wayland, they aren't. This commit adds a
property that allows to explicitly constrain popovers to
the toplevel, giving them the same behavior under Wayland
as under X11.

https://bugzilla.gnome.org/show_bug.cgi?id=757474

filechooserbutton: add some spacing between icon and text

These were a bit crammed, which doesn't match well with spacing used in
various other parts of gtk.

gtkwindow: remove headerbar after disposing parent

Widgets such as gtkfilechooser may be saving their size and position on
the unmap callback, if the client-side decoration header bar is removed
first, the reported size will be wrong.

https://bugzilla.gnome.org/show_bug.cgi?id=756618

wayland: re-configure surface even if size matches

gdk-wayland backend would not re-configure a surface when its size and
scale match the known size and scale.

But there might be a pending xdg_surface_configure() that would revert
this change so we should re-configure even if the currently known
size/scale match, otherwise we may end up with a wrong size after the
xdg_surface_configure() is received.

Bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=758901

wayland: Invert drop_finished/button release call order

If we "release" the button first, the drag will be eventually cancelled,
we must first signal GDK_DROP_FINISHED, and then release the button so
the success status prevails.

wayland: Store requested target on GdkWaylandSelection

It doesn't make a lot of sense to have this stored as data offer data,
rather together with the source_targets array, which is what we're
poking here in the end.

https://bugzilla.gnome.org/show_bug.cgi?id=758713

Add some more missing nullable annotations

GtkBox: The center widget can be NULL

Propagates to GtkActionBar.

GtkWidget: Add some missing nullable annotations

inspector: Show the size request mode

This is not a property, so show it in the misc page.

window: Use g_set_object

No need to do the same thing manually.

testentryicons: update dnd example

This code was #ifdefed out since 2008. Time to try it.

testentryicons: Use an existing icon name

dialog-info is not an existing icon.

wayland: Refactor selection ownership

Dissociate ownership from our maintenance of wl_data_source objects.
The only place where ownership must be updated together is
data_source.cancelled, for the other places GDK should take care of
setting up the right ownership, even if at a different order than
we'd expect here.

This fixes GTK+ apps on wayland being locally confused about the
current selection ownership. Because gtk_selection_add_targets()
results in a wl_data_source being created, and ownership being
updated right away, early callers of this will change the ownership
even if the widget it's being called on didn't explicitly request
the selection ownership yet.

https://bugzilla.gnome.org/show_bug.cgi?id=758660

textiter: fix bug in case insensitive backward search

'win.lines' contains the same content as the GtkTextBuffer, so to find
@match_start, forward_chars_with_skipping() is called with
skip_decomp=FALSE (the last parameter). So far so good.

On the other hand, the content 'lines' (the needle split in lines) is
casefolded and normalized for a case insensitive search. So,
forward_chars_with_skipping(..., skip_decomp=TRUE) must be called only
for the portion of text containing the needle.

Since 'start_tmp' contains the location at the start of the match, we
can simply begin at that location to find the end of the match.

Unit tests are added.

https://bugzilla.gnome.org/show_bug.cgi?id=758698

listbox: Use :focus-on-click policy from row instead of list

Using the property from the container is inconsistent with the
:selectable and :activatable properties, which are per row.

icon-browser: Stop using GtkMisc properties

Instead of xpad/ypad of 4 for every widget, just increase the
row-spacing and column-spacing by 8.

https://bugzilla.gnome.org/show_bug.cgi?id=758790

icon-browser: Use proper "times" character

https://bugzilla.gnome.org/show_bug.cgi?id=758790

icon-browser: baseline-align the icon size labels

https://bugzilla.gnome.org/show_bug.cgi?id=758790

stack: Update windows before allocating children

Doing things the other way around seems to cause problems in
some cases where children want to do different things depending
on the window position.

https://bugzilla.gnome.org/show_bug.cgi?id=758563

scalebutton: Annotate the return type

Instead of just listing the return type of get_plus_button() and
get_minus_button() in the documentation, we can use the (type)
annotation to ensure that the introspection data reflects the actual
type of the returned widget.

gtkwindow: fix regression with firefox dropdown menu

Fix a regression introduced by:

commit 6866d1c widget: Make gtk_widget_queue_allocate() not resize

Where the dropdown menu in Firefox would not be relocated after the
toplevel window is moved.

bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=758609